Skip to content

Conversation

@julien-lang
Copy link
Member

@julien-lang julien-lang commented Dec 12, 2025

Simplify the os_version/python_version combination for calling the test template.

No longer use a matrix strategy because only work with one dimension.
Define and use parameter for each version dimension and use the each operator for looping over both parameters.

Replicated changes on multiple repositories

@julien-lang julien-lang changed the title Tests SG-40980 Simplify Azure Pipeline CI pipeline Dec 12, 2025
@julien-lang julien-lang requested a review from Copilot December 16, 2025 19:15
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR refactors the Azure Pipeline CI configuration to simplify test execution across multiple OS and Python version combinations. The changes replace a repetitive, manually-defined job structure with a more maintainable loop-based approach using Azure DevOps' each operator.

Key Changes:

  • Introduced parameterized VFX Reference Platform versions (CY2022-2024) with associated Python and Qt wrapper configurations
  • Consolidated OS version definitions into a single parameter with VM image mappings
  • Replaced 9 individual job template invocations with nested loops that dynamically generate test jobs

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
internal/run-tests.yml Refactored from explicit job definitions to parameterized loops using VFX platform and OS version parameters
internal/run-tests-with.yml Updated default parameter values and simplified job display name to accommodate dynamic generation

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@julien-lang julien-lang requested a review from a team December 16, 2025 20:34
@julien-lang julien-lang merged commit b9df48b into master Jan 5, 2026
22 checks passed
@julien-lang julien-lang deleted the ticket/SG-40980-simplify-ci-pipeline branch January 5, 2026 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants